Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CSG-1097: Update Comet Starter to better support theming #30

Merged
merged 5 commits into from
Oct 10, 2023
Merged

CSG-1097: Update Comet Starter to better support theming #30

merged 5 commits into from
Oct 10, 2023

Conversation

jbouder
Copy link
Collaborator

@jbouder jbouder commented Oct 10, 2023 •
edited
Loading

Updates to baseline app to utilize USWDS SCSS for styling to support USWDS theming and design token use.

Description

  • Updated to latest comet-uswds version (does not include USWDS css)
  • Updated app config to load USWDS package
  • Updated uswds settings to load font and image assets from USWDS package
  • Removed unneeded assets from app
  • Added base set of theme design tokens to USWDS Settings SCSS file

Related Issue

N/A

Motivation and Context

  • Updates to support use of USWDS theming
  • Resolve issue where USWDS css overrides must be set to !important

How Has This Been Tested?

  • Verified Local Testing
  • Verified Production Build and Deployment through docker

Screenshots (if appropriate):

Screenshot 2023-10-10 at 8 48 31 AM

@jbouder jbouder added enhancement New feature or request refactor Misc changes to project structure or configuration labels Oct 10, 2023
@jbouder jbouder merged commit 7aac780 into main Oct 10, 2023
5 checks passed
@jbouder jbouder deleted the CSG-1097 branch October 10, 2023 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request refactor Misc changes to project structure or configuration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jbouder